Fix output formats for the `theme mod get` command by philipjohn · Pull Request #456 · wp-cli/extension-command

philipjohn · 2025-06-05T14:04:01Z

Fixes #96

I maintained but improved the space-indented formatting for the table format as folks might be using this (as per @schlessera's comment) and then fixed the other formats by using dot notation to represent each field, given the formatter needs arrays with key and value.

For the table format, I opted to remove the => for the value to represent an array. I felt it was harder to read than simply leaving the value blank and letting the indent do the visual work.

For all formats booleans and empty arrays are represented by a string (e.g. [empty array]. There may be a better way to represent these, and I did consider var_export() but others may have better ideas.

When specific mods are requested but don't exist, they are included in the output with null as the value except for CSV output which just has an empty string.

New Behat tests are included for each of the formats (table, csv, json, yaml).

Here are some examples of the new output:

$ wp eval 'print_r(get_theme_mods());'
Array
(
    [0] => 
    [nav_menu_locations] => Array
        (
        )

    [sidebars_widgets] => Array
        (
            [time] => 1749114163
            [data] => Array
                (
                    [wp_inactive_widgets] => Array
                        (
                            [0] => block-2
                            [1] => block-3
                            [2] => block-4
                            [3] => block-5
                            [4] => block-6
                        )

                    [sidebar-1] => Array
                        (
                            [0] => block-7
                        )

                )

        )

)

$ wp theme mod get --all --format=table
+-------------------------+---------------+
| key                     | value         |
+-------------------------+---------------+
| 0                       | [false]       |
| nav_menu_locations      | [empty array] |
| sidebars_widgets        |               |
|   time                  | 1749114163    |
|   data                  |               |
|     wp_inactive_widgets |               |
|       0                 | block-2       |
|       1                 | block-3       |
|       2                 | block-4       |
|       3                 | block-5       |
|       4                 | block-6       |
|     sidebar-1           |               |
|       0                 | block-7       |
+-------------------------+---------------+

$ wp theme mod get --all --format=csv
key,value
0,[false]
nav_menu_locations,"[empty array]"
sidebars_widgets.time,1749114163
sidebars_widgets.data.wp_inactive_widgets.0,block-2
sidebars_widgets.data.wp_inactive_widgets.1,block-3
sidebars_widgets.data.wp_inactive_widgets.2,block-4
sidebars_widgets.data.wp_inactive_widgets.3,block-5
sidebars_widgets.data.wp_inactive_widgets.4,block-6
sidebars_widgets.data.sidebar-1.0,block-7

$ wp theme mod get --all --format=yaml
---
- 
  key: "0"
  value: '[false]'
- 
  key: nav_menu_locations
  value: '[empty array]'
- 
  key: sidebars_widgets.time
  value: 1749114163
- 
  key: sidebars_widgets.data.wp_inactive_widgets.0
  value: block-2
- 
  key: sidebars_widgets.data.wp_inactive_widgets.1
  value: block-3
- 
  key: sidebars_widgets.data.wp_inactive_widgets.2
  value: block-4
- 
  key: sidebars_widgets.data.wp_inactive_widgets.3
  value: block-5
- 
  key: sidebars_widgets.data.wp_inactive_widgets.4
  value: block-6
- 
  key: sidebars_widgets.data.sidebar-1.0
  value: block-7

$ wp theme mod get --all --format=json | jq .
[
  {
    "key": "0",
    "value": "[false]"
  },
  {
    "key": "nav_menu_locations",
    "value": "[empty array]"
  },
  {
    "key": "sidebars_widgets.time",
    "value": 1749114163
  },
  {
    "key": "sidebars_widgets.data.wp_inactive_widgets.0",
    "value": "block-2"
  },
  {
    "key": "sidebars_widgets.data.wp_inactive_widgets.1",
    "value": "block-3"
  },
  {
    "key": "sidebars_widgets.data.wp_inactive_widgets.2",
    "value": "block-4"
  },
  {
    "key": "sidebars_widgets.data.wp_inactive_widgets.3",
    "value": "block-5"
  },
  {
    "key": "sidebars_widgets.data.wp_inactive_widgets.4",
    "value": "block-6"
  },
  {
    "key": "sidebars_widgets.data.sidebar-1.0",
    "value": "block-7"
  }
]

schlessera

Looks good, only minor nitpicks for the code. Also, it would be good to have at least 1 feature test covering this.

philipjohn · 2025-06-06T08:42:30Z

Oh, I will do tests by the way! Just probably not until after this week :)

Copilot

Pull request overview

This PR refactors the wp theme mod get implementation to produce well-structured output across table, CSV, JSON, and YAML formats, particularly for nested option values, addressing malformed output reported in issue #96.

Changes:

Introduces a recursive mod_to_string() helper to flatten nested theme mod arrays/objects into a list of { key, value } pairs, with special handling for booleans and empty arrays.
Adjusts table formatting to use indentation for nested keys and uses dot-notation for hierarchical keys in CSV/JSON/YAML formats, while filtering out items deemed to have “no value” in non-tabular formats.
Updates usage of WP_CLI\Utils\get_flag_value() via an imported Utils alias and reuses the new formatter logic for both get and list_ subcommands.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-02-03T01:29:47Z

+		// If specific mods are requested, filter out any that aren't requested.
+		$mods = ! empty( $args ) ? array_intersect_key( get_theme_mods(), array_flip( $args ) ) : get_theme_mods();

-		// For unset mods, show blank value.
-		foreach ( $args as $mod ) {
-			if ( ! isset( $mods[ $mod ] ) ) {
-				$list[] = [
-					'key'   => $mod,
-					'value' => '',
-				];
+		// Generate the list of items ready for output. We use an initial separator that we can replace later depending on format.
+		$separator = '\t';
+		array_walk(
+			$mods,
+			function ( $value, $key ) use ( &$mod_list, $separator ) {


The previous implementation added rows for requested mods that are not present in get_theme_mods() (see the old "For unset mods, show blank value." logic), which is relied on by the theme-mod.feature example for header_textcolor. With this new $mods construction and no subsequent pass over $args, requested-but-unset mods are now silently omitted from the output for all formats, changing the CLI behavior; to preserve backward compatibility you should add entries with an empty value for any requested mod that is not present in $mods (at least for the table format, and optionally for others).

Copilot · 2026-02-03T01:29:48Z

+			// For JSON, CSV, and YAML formats we use dot notation to show the hierarchy.
+			case 'csv':
+			case 'yaml':
+			case 'json':
+				$mod_list = array_filter(
+					array_map(
+						static function ( $item ) use ( $separator ) {
+							return [
+								'key'   => str_replace( $separator, '.', $item['key'] ),
+								'value' => $item['value'],
+							];
+						},
+						$mod_list
+					),
+					function ( $item ) {
+						return ! empty( $item['value'] );
+					}
+				);
+				break;


The new flattening and dot-notation logic for CSV/YAML/JSON formats introduces non-trivial behavior but currently has no dedicated acceptance tests (unlike features/theme-mod-list.feature, which covers those formats for the list subcommand). Given that issue #96 was caused by malformed non-tabular output, it would be valuable to add Behat coverage for wp theme mod get --all --format=csv|json|yaml with nested theme-mod structures to ensure this formatter behavior is exercised and guarded against regressions.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Co-authored-by: Pascal Birchler <pascal.birchler@gmail.com>

swissspidy · 2026-04-08T15:55:22Z

This is lacking tests & is breaking existing ones. Will see if I can pick this up to bring it over the finish line.

codecov · 2026-06-04T11:01:49Z

Codecov Report

❌ Patch coverage is 78.33333% with 13 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
src/Theme_Mod_Command.php	78.33%	13 Missing ⚠️

📢 Thoughts on this report? Let us know!

philipjohn added 4 commits June 5, 2025 15:38

96: Fix output formats for the theme mod get command

bc6035d

96: Handle both booleans

84891e7

96: Handle objects

d80db8f

96: Fix formatting and rename variable to please the CS Gods

e354614

philipjohn requested a review from a team as a code owner June 5, 2025 14:04

schlessera requested changes Jun 5, 2025

View reviewed changes

Comment thread src/Theme_Mod_Command.php Outdated

Comment thread src/Theme_Mod_Command.php Outdated

Comment thread src/Theme_Mod_Command.php Outdated

Comment thread src/Theme_Mod_Command.php Outdated

philipjohn added 5 commits June 6, 2025 10:08

96: Remove unnecessary array check already done by get_theme_mods()

561cfc8

96: Use tab character as separator

4b0aa30

96: Import Utils for much more handsome code

26b9aff

96: Make the array_map callbacks static

5864145

96: Set the separator once

76025d7

philipjohn requested a review from schlessera June 6, 2025 08:14

github-actions Bot added command:theme-mod-get Related to 'theme mod get' command scope:distribution Related to distribution scope:testing Related to testing labels Dec 22, 2025

Merge branch 'main' into fix/96-theme-mod-get-format

efb8947

swissspidy requested a review from Copilot February 3, 2026 01:23

Copilot started reviewing on behalf of swissspidy February 3, 2026 01:23 View session

Copilot AI reviewed Feb 3, 2026

View reviewed changes

swissspidy and others added 2 commits March 12, 2026 13:39

Update src/Theme_Mod_Command.php

97f2985

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Merge branch 'main' into fix/96-theme-mod-get-format

01b24ef

swissspidy reviewed Mar 19, 2026

View reviewed changes

Comment thread src/Theme_Mod_Command.php Outdated

swissspidy and others added 2 commits March 19, 2026 13:42

Apply suggestions from code review

2d70f93

Co-authored-by: Pascal Birchler <pascal.birchler@gmail.com>

Merge branch 'main' into fix/96-theme-mod-get-format

5e0ba2a

swissspidy added the state:unconfirmed label Apr 8, 2026

philipjohn added 2 commits June 4, 2026 11:21

96: Restore all theme mod keys passed in via arguments

f3bbcec

96: Add missing space

11a8de3

philipjohn added 2 commits June 4, 2026 13:38

96: Ensure empty values are included in all output formats

6cbe267

96: Add Behat tests covering all formats for theme get mod

eff2914

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix output formats for the `theme mod get` command#456

Fix output formats for the `theme mod get` command#456
philipjohn wants to merge 18 commits into
wp-cli:mainfrom
philipjohn:fix/96-theme-mod-get-format

philipjohn commented Jun 5, 2025 •

edited

Loading

Uh oh!

schlessera left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

philipjohn commented Jun 6, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Feb 3, 2026

Uh oh!

Uh oh!

Uh oh!

Copilot AI Feb 3, 2026

Uh oh!

Uh oh!

swissspidy commented Apr 8, 2026

Uh oh!

codecov Bot commented Jun 4, 2026 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

philipjohn commented Jun 5, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

schlessera left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

philipjohn commented Jun 6, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Feb 3, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Copilot AI Feb 3, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

swissspidy commented Apr 8, 2026

Uh oh!

codecov Bot commented Jun 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

philipjohn commented Jun 5, 2025 •

edited

Loading

codecov Bot commented Jun 4, 2026 •

edited

Loading